Skip to content

fix(caps-warning): caps state isn't always correct (@Leonabcd123, @fehmer)#7478

Merged
Miodec merged 45 commits intomonkeytypegame:masterfrom
Leonabcd123:fix-caps-warning
Feb 27, 2026
Merged

fix(caps-warning): caps state isn't always correct (@Leonabcd123, @fehmer)#7478
Miodec merged 45 commits intomonkeytypegame:masterfrom
Leonabcd123:fix-caps-warning

Conversation

@Leonabcd123
Copy link
Contributor

@Leonabcd123 Leonabcd123 commented Feb 6, 2026

Description

Account for platform specific behavior.

Fixes #7472

@monkeytypegeorge monkeytypegeorge added the frontend User interface or web stuff label Feb 6, 2026
@Leonabcd123 Leonabcd123 changed the title fix(caps-warning): caps warning is inverted when caps is active on page load (@Leonabcd123) fix(caps-warning): caps warning is inverted when caps is active on page load (@fehmer, @Leonabcd123) Feb 6, 2026
@VardanRattan
Copy link

This works, but it does reintroduce internal inversion for Linux CapsLock events.

Conceptually, the issue on Linux is timing (modifier state not settled at event time), not that getModifierState("CapsLock") is incorrect. A deferred read would avoid platform-specific inversion and keep CapsLock state fully derived from OS state.

Might be worth considering if we want to avoid encoding Linux timing assumptions into logic long-term.

@Leonabcd123 Leonabcd123 changed the title fix(caps-warning): caps warning is inverted when caps is active on page load (@fehmer, @Leonabcd123) fix(caps-warning): caps state isn't always correct (@Leonabcd123, @fehmer) Feb 26, 2026
@socket-security
Copy link

socket-security bot commented Feb 27, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Added@​leonabcd123/​modern-caps-lock@​2.0.370100769270

View full report

@github-actions
Copy link
Contributor

Continuous integration check(s) failed. Please review the failing check's logs and make the necessary changes.

@github-actions github-actions bot added the waiting for update Pull requests or issues that require changes/comments before continuing label Feb 27, 2026
@github-actions github-actions bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label Feb 27, 2026
@github-actions
Copy link
Contributor

Continuous integration check(s) failed. Please review the failing check's logs and make the necessary changes.

@github-actions github-actions bot added the waiting for update Pull requests or issues that require changes/comments before continuing label Feb 27, 2026
@github-actions github-actions bot removed the waiting for update Pull requests or issues that require changes/comments before continuing label Feb 27, 2026
@Miodec Miodec merged commit ba4d4e9 into monkeytypegame:master Feb 27, 2026
13 checks passed
@Leonabcd123 Leonabcd123 deleted the fix-caps-warning branch February 27, 2026 14:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

frontend User interface or web stuff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Caps Lock indicator becomes inverted when page is opened with Caps Lock already enabled

4 participants